From ec7ccabe282921945fca3d164f3b35f1fa05950a Mon Sep 17 00:00:00 2001 From: =?utf8?q?Florian=20M=C3=BCllner?= Date: Fri, 25 Apr 2025 14:48:23 +0200 Subject: [PATCH] workspace: Simplify hasMaximizedWindows() Since mutter exposes `is_maximized()` now, use that instead of checking for both horizontal and vertical maximization. Part-of: --- js/ui/workspace.js | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/js/ui/workspace.js b/js/ui/workspace.js index adbecb28b..25c39aa5d 100644 --- a/js/ui/workspace.js +++ b/js/ui/workspace.js @@ -1273,14 +1273,11 @@ class Workspace extends St.Widget { // check for maximized windows on the workspace hasMaximizedWindows() { - for (let i = 0; i < this._windows.length; i++) { - let metaWindow = this._windows[i].metaWindow; - if (metaWindow.showing_on_its_workspace() && - metaWindow.maximized_horizontally && - metaWindow.maximized_vertically) - return true; - } - return false; + return this._windows.some(w => { + const {metaWindow} = w; + return metaWindow.showing_on_its_workspace() && + metaWindow.is_maximized(); + }); } _clearSkipTaskbarSignals() { -- 2.39.5